Skip to content

feat: add pushToTop and pushToTopOnce#173

Open
bnb wants to merge 2 commits intoedge-js:6.xfrom
bnb:bnb/add-pushToTop-and-pushToTopOnce
Open

feat: add pushToTop and pushToTopOnce#173
bnb wants to merge 2 commits intoedge-js:6.xfrom
bnb:bnb/add-pushToTop-and-pushToTopOnce

Conversation

@bnb
Copy link

@bnb bnb commented Feb 13, 2026

🔗 Linked issue

#172

❓ Type of change

  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This PR adds pushToTop and pushToTopOnce and adds tests for pushToTop, mirroring pushTo's tests. I've run the tests locally and it works.

I've got a situation where I have a list of events in the future and in the past, and I want the ones in the past to be in reverse chronological order but the ones in the future to be in chronological order. Rather than having to do that logic in JS, it'd be super nice to just be able to do that with Edge. pushToTop makes it super simple to set the events in the past in reverse chronological order without any JS logic around this in my app.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

bnb added 2 commits February 13, 2026 13:30
Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
@bnb
Copy link
Author

bnb commented Feb 13, 2026

I'll work on a docs PR now ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant